thanks for implementing this improvement! i'll align our ci setup to these changes and eventually, i'll report back with suggested improvements.

Thanks for considering my MR!

I've been successfully using my branch in a small project in which I wanted to write a dart only CLI to use on desktop.

One small issue I see is the generated code depends on sqfliteDatabaseFactory which needs to be initialized differently if using floor (flutter) or floor_ffi (dart only). The implication is the code declaring the database has to import one or the other. As a result, the database class cannot be added to a common package imported by both a flutter app or a dart only cli app. This doesn't seem to be a big issue IMHO (an abstract base class gets most of the job done), but just something for you to be aware.

If you have any concerns, I'll be happy to update this MR. Or feel free to use my MR as you see fit in case you want to use it as a basis to do your own MR.

Thanks!

@vitusortner Could you please . merge this pull request

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.31%. Comparing base (4146037) to head (28d05d9).
Report is 1 commits behind head on develop.

❗ Current head 28d05d9 differs from pull request most recent head 435813a. Consider uploading reports for the commit 435813a to get more accurate results

@@             Coverage Diff             @@
##           develop     #638      +/-   ##
===========================================
+ Coverage    91.51%   94.31%   +2.79%     
===========================================
  Files           11       10       -1     
  Lines          224      211      -13     
===========================================
- Hits           205      199       -6     
+ Misses          19       12       -7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

I see this PR checks in the generated code now. We did not have this before and they were git ignored, but only under /floor. Now that all this lives under floor_common, the gitignore must be updated. @hendrikvdkaaden Can you remove the generated files from git and after that update the gitignore file?